Skip to content

Conversation

@ReevesAk
Copy link

The README.md is updated to properly guide the installation and running of tinydecred.

Comment on lines +11 to +12
.idea/
.vscode/
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For folders specific to your own environment, please use .git/info/exclude instead.

Comment on lines +17 to +23
To run tinydecred on your machine you will need to set up the following.

- pip install websocket_client
- pip install blake256
- pip install base58
- pip install PyNaCl
- pip install appdirs
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You shouldn't install this way. Regular users should just pip install decred.

For tinywallet, you can poetry install and then poetry run tinywallet, but you'll need the fix from #198.

Comment on lines +25 to +26
Set up and run dcrd. Ensure your rpcusername and rpcpassword is set in your dcrd config file.
Installation guide can be found [`here`](https://docs.decred.org/wallets/cli/dcrd-setup/).
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dcrd is completely optional, and it's integration is very shallow right now. I wouldn't say much about dcrd yet.

Comment on lines -16 to +18
from decred import DecredError
from decred.util import encode
from decred.util.encode import ByteArray, unblobCheck
from decred.decred import DecredError
from decred.decred.util import encode
from decred.decred.util.encode import ByteArray, unblobCheck
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can't do this. Typically this would indicate that you are importing the package in a way that it wasn't intended. How are you using the package that requires this change?

Copy link
Author

@ReevesAk ReevesAk Jun 13, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When I run tinywallet, without this change I was getting an importerror(unknown location), so I figured since it is absolute import, this would help. The only other fix I could figure out was the insertion of my projectpath using sys.path.insert().
This was what I used to make tinywallet run on my machine, I would be glad if you can proffer a better way.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are you using the CLI command poetry run tinywallet to run tinywallet?

Copy link
Member

@buck54321 buck54321 Jun 13, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You'll need to pip install poetry and poetry install in the tinywallet directory first.

Sorry this stuff isn't better documented already.

Copy link
Author

@ReevesAk ReevesAk Jun 21, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have implemented your suggestions, and I still got the error I dropped on tinydecred room..
what version of python are you using?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are you referring to the IDE error?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, it's actually not IDE related.

Copy link
Member

@JoeGruffins JoeGruffins Jun 23, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Neither buck nor I are seeing this, and we use different flavors of linux I believe. I'm not great with poetry, but It looks like the virtual environment it uses isn't being set up properly on your end.


## Requirements

To run tinydecred on your machine you will need to set up the following.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You don't "run tinydecred". You can run tinywallet, or use the decred package.

@ReevesAk ReevesAk closed this Jun 28, 2021
@ReevesAk
Copy link
Author

Issue was resolved by fix from #198.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants